Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

Zp patch ethers impl#38

Merged
thodges-gh merged 4 commits intoethers-implementationfrom
zp-patch-ethers-impl
Sep 12, 2025
Merged

Zp patch ethers impl#38
thodges-gh merged 4 commits intoethers-implementationfrom
zp-patch-ethers-impl

Conversation

@zeuslawyer
Copy link
Copy Markdown
Contributor

@zeuslawyer zeuslawyer commented Sep 12, 2025

Hey @thodges-gh please check the DM. integration tests are not working well and need to be fixed before PR35 is ready for merge. I did updates etc to front end, and put in place test suite and also made refactors for readability to adapter/ethers.ts

…at accepts

Ethers.js signer.

Update example nextjs app's routes to show wagmi use and etherjs use with ccipClient.

Update dependencies

- Updated `@chainlink/ccip-js` dependency in `examples/nextjs/package.json` to use workspace reference. This means it will use
local built version of ccip-js package rather than published version. But will use published version when it itself is publised
to npmjs.
- Added new Ethers.js components and page routes to show the CCIP JS Package API.
- Updated `pnpm-lock.yaml` to reflect new dependencies and versions.
- Removed outdated Babel dependencies and updated node types across various packages.
…rrors.

Refactor Ethers.js adapter to use custom type definitions for contract calls, transactions, receipts, and logs. This change eliminates 'as any' casts, improving type safety and code clarity. Additionally, update integration tests to reflect these changes and ensure compatibility with the new type definitions.
Copy link
Copy Markdown
Contributor

@thodges-gh thodges-gh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to push some small fixes but now it works for me. Going to merge and work off the target branch.

@thodges-gh thodges-gh merged commit d1dd9c5 into ethers-implementation Sep 12, 2025
2 checks passed
@thodges-gh thodges-gh deleted the zp-patch-ethers-impl branch September 12, 2025 11:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants